Skip to content

Refactor: Rename EXCLUDE_FORKED_REPOS to EXCLUDE_CONTRIB_REPOS#136

Closed
koichiio wants to merge 1 commit intojstrieb:masterfrom
koichiio:master
Closed

Refactor: Rename EXCLUDE_FORKED_REPOS to EXCLUDE_CONTRIB_REPOS#136
koichiio wants to merge 1 commit intojstrieb:masterfrom
koichiio:master

Conversation

@koichiio
Copy link
Copy Markdown

@koichiio koichiio commented Nov 4, 2025

The previous variable name was misleading. It doesn't just exclude forked repositories - it excludes all repositories the user has contributed to but doesn't own (whether they're forks or original projects).

Changes:

  • Rename EXCLUDE_FORKED_REPOS → EXCLUDE_CONTRIB_REPOS
  • Rename ignore_forked_repos → exclude_contrib_repos in Python code
  • Add backward compatibility in generate_images.py with deprecation warning
  • Update README.md with clearer documentation explaining the behavior

The new name accurately reflects what the variable does: exclude contributed repositories from statistics calculation.

The previous variable name was misleading. It doesn't just exclude forked
repositories - it excludes all repositories the user has contributed to but
doesn't own (whether they're forks or original projects).

Changes:
- Rename EXCLUDE_FORKED_REPOS → EXCLUDE_CONTRIB_REPOS
- Rename ignore_forked_repos → exclude_contrib_repos in Python code
- Add backward compatibility in generate_images.py with deprecation warning
- Update README.md with clearer documentation explaining the behavior

The new name accurately reflects what the variable does: exclude contributed
repositories from statistics calculation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@koichiio koichiio closed this Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant